Skip to content

Mc error logging - #66

Merged
dankessler merged 3 commits into
UMPsychMethodsCore:developfrom
mangstad:mc_Error_logging
Jun 19, 2012
Merged

Mc error logging#66
dankessler merged 3 commits into
UMPsychMethodsCore:developfrom
mangstad:mc_Error_logging

Conversation

@mangstad

Copy link
Copy Markdown
Contributor

Added a logging function to matlabScripts. mc_Logger has two functions:

the setup function takes a log directory and creates a timestamped copy of the original calling script (usually _mc_template script) in the logfile directory with a header that includes the current release tag and SHA ID (if available) and also creates a .log file with the same name/timestamp for logging purposes.

the log function takes a string and writes it to the .log file in the log directory.

@mangstad

Copy link
Copy Markdown
Contributor Author

This should resolve issues #27 and #28 when accepted.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to be clear, this was removed since path adding happens in the template script now, right?

@dankessler

Copy link
Copy Markdown
Member

As anticipated, looks like this will no longer cleanly apply. I'll do the merge in my repository and try to clean up any conflicts.

@dankessler
dankessler merged commit 8e986d3 into UMPsychMethodsCore:develop Jun 19, 2012
@dankessler

Copy link
Copy Markdown
Member

The merge was not actually so bad, and the conflicts were pretty easy to figure out (only one per file). It's now merged in. That said, we'll continue to follow the plan outlined in #43 and do testing on this palimpsest of a repository :)

This was referenced Jun 19, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants